Conversation
Summary of ChangesHello @minngyuseong, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 PR은 사용자가 특정 여행 경비 세부 정보를 볼 수 있도록 여행 상세 페이지 구현을 도입합니다. 또한, 일반 여행 개요와 동적 ID를 사용하는 개별 여행 항목에 대한 상세 보기를 모두 지원하도록 여행 페이지의 라우팅을 리팩토링합니다. Highlights
Changelog
Activity
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
이 PR은 여행 상세 페이지를 구현하고 관련 라우팅을 설정합니다. 새로운 TravelDetailPage 컴포넌트는 더미 데이터를 사용하여 지출 내역을 보여주는 데이터 테이블을 포함하고 있습니다. 전반적으로 페이지의 기본 구조는 잘 잡혀있으나, UI에 표시되는 텍스트의 잠재적인 중복 문제를 해결하는 것이 좋겠습니다. 렌더링 성능 최적화는 현재 더미 데이터 사용으로 인해 우선순위가 낮지만, 향후 API 연동 시 고려해볼 수 있습니다. 자세한 내용은 코드 리뷰 주석을 참고해주세요.
minngyuseong
force-pushed
the
feat/#396/travel-detail-page
branch
from
dfedcf6 to
ec0074d
Compare
- 여행지 Title - 지출 내역 불러오기 BottomSheet 반영
minngyuseong
force-pushed
the
feat/#396/travel-detail-page
branch
from
ec0074d to
da1b66e
Compare
Kjiw0n
left a comment
Kjiw0n
left a comment
There was a problem hiding this comment.
수고하셨습니다 👍 👍 여행 페이지까지 빠르게 작업해주셨네요
코멘트 확인 부탁드려요!!
There was a problem hiding this comment.
해당 컴포넌트를 띄워보니까 밑에 약간 잘리는데,, 확인 부탁드려요.
There was a problem hiding this comment.
이거 z값 문제라서 다른 issue에서 전체적으로 z값 통일 작업중입니다. 이번 pr에서는 넘어갈게요!
- table 관련 태그는 flex 설정을 적용하면 자체적으로 너비를 설정해주는 속성이 모두 무시되므로 wrapper div에 원하는 설정 기입 - header는 sticky 설정
- children에서 자체적으로 scroll하기 위함
Kjiw0n
left a comment
Kjiw0n
left a comment
There was a problem hiding this comment.
- UI가 디자인과 다른 부분에 빨간색 박스로 표시해두었어요. 반영 부탁드립니다!
|
넵 확인했습니다!! 변경하신 방향이 더 맞는 방향인 것 같네요 👍 |
2 participants
관련 이슈
closes #396
작업 내용
여행 상세 페이지를 만들었어요.
지출내역 추가하기 버튼을 누르면 BottomSheet가 올라와서 가계부에서 지출내역을 선택할 수 있는 UI만들었어요.
스크린샷
💬 공유사항 to 리뷰어
BottomSheet에서 스크롤될때 TableHead가 고정돼야하는데, 전체적으로 z값을 정리할 필요가 있어서 다른 PR에서 z값 전부 통일하고 sticky 설정할게요.
필수 체크사항